Skip to content

Option A: Budget-Based Restocking Tool#39

Open
jaichenchlani wants to merge 5 commits intobeck-source:mainfrom
jaichenchlani:main
Open

Option A: Budget-Based Restocking Tool#39
jaichenchlani wants to merge 5 commits intobeck-source:mainfrom
jaichenchlani:main

Conversation

@jaichenchlani
Copy link
Copy Markdown

No description provided.

jaichenchlani and others added 5 commits April 7, 2026 10:35
Updated the GitHub MCP server configuration to use the GHTOKEN
environment variable for authentication instead of
GITHUB_PERSONAL_ACCESS_TOKEN.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
New features | Option A: Budget-Based Restocking Tool
Fixed 10 critical bugs in the Reports page to bring it up to the same quality standards as other pages:

**Translation Issues:**
- Added complete i18n support using t() function for all text
- Added Japanese translations for all Reports page strings
- Page now translates properly when switching languages

**Global Filter Integration:**
- Integrated useFilters() composable to respect global filters
- Added watch() to reload data when filters change
- Reports now filter by warehouse, category, status, and time period

**Code Quality:**
- Removed all console.log() debug statements (10+ removed)
- Converted from Options API to Composition API for consistency
- Replaced var with const/let throughout
- Uses centralized api client instead of direct axios calls
- Added proper API endpoints: getQuarterlyReports() and getMonthlyTrends()

**Data Display:**
- Fixed v-for keys to use unique identifiers instead of array index
- Added multi-currency support ($ for USD, ¥ for JPY)
- Uses formatCurrency() utility for proper locale-aware formatting
- Month names now use proper locale formatting

All bugs verified fixed through browser testing. Page now works consistently with Dashboard, Orders, and other views.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant